New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add 'orEmpty' convenience function to Option #58

Merged

millyrowboat merged 1 commit into main from milly/20230912-orempty

Sep 20, 2023

Collaborator

millyrowboat commented Sep 13, 2023

Clawed back from #53

orEmpty is a convenience function that we've been using in our codebases, that will return an empty string if your Option is None
- Wondering now if that should only be allowed to be called on a Option context. Thoughts? As in:

fun Option<String>.orEmpty(f: (String) -> String): String = this.map(f).getOrElse { "" }


          Add 'orEmpty' convenience function to Option

1b966fb

millyrowboat requested review from Synesso, hugomd and cwmyers as code owners

September 13, 2023 07:34

hugomd approved these changes

View reviewed changes

cwmyers approved these changes

View reviewed changes

millyrowboat merged commit a65ef0d into main

2 checks passed

millyrowboat deleted the milly/20230912-orempty branch

September 20, 2023 23:46

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet